Skip to content

snapcraft.yaml: comment out isa-riscv64-rva23#437

Merged
alfonsosanchezbeato merged 1 commit intocanonical:mainfrom
alfonsosanchezbeato:workaround-for-riscv-assumes
Apr 23, 2026
Merged

snapcraft.yaml: comment out isa-riscv64-rva23#437
alfonsosanchezbeato merged 1 commit intocanonical:mainfrom
alfonsosanchezbeato:workaround-for-riscv-assumes

Conversation

@alfonsosanchezbeato
Copy link
Copy Markdown
Member

Comment out isa-riscv64-rva23 assumes. Our github build process will uncomment it only for riscv64 architecture. This is needed because snapd versions previous to 2.75 fail to install core26 if the flag is present, for any architecture. And in many cases the snapd present is the one from the debian package.

Comment out isa-riscv64-rva23 assumes. Our github build process will uncomment
it only for riscv64 architecture. This is needed because snapd versions
previous to 2.75 fail to install core26 if the flag is present, for any
architecture. And in many cases the snapd present is the one from the debian
package.
Comment thread snapcraft.yaml
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the snapcraft.yaml base snap metadata to avoid declaring an assumes capability that breaks installation on older snapd versions, while keeping a path for enabling it on riscv64 in CI.

Changes:

  • Comments out the isa-riscv64-rva23 entry under assumes.
  • Adds an explanatory comment indicating CI will enable it for riscv64 builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread snapcraft.yaml
Comment thread snapcraft.yaml
@valentindavid valentindavid self-requested a review April 23, 2026 14:03
@alfonsosanchezbeato alfonsosanchezbeato merged commit 828b1ca into canonical:main Apr 23, 2026
16 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants